Code cleanup - sorted imports and python code style guides#126
Open
xidus90 wants to merge 2 commits intoAIrjen:mainfrom
Open
Code cleanup - sorted imports and python code style guides#126xidus90 wants to merge 2 commits intoAIrjen:mainfrom
xidus90 wants to merge 2 commits intoAIrjen:mainfrom
Conversation
Owner
|
Hi! Thank you, this is very interesting. As you can tell, I’m not a software developer 😅 I am hesitant to merge this right now. I currently don’t have computer access. Just my phone. Meaning that I have no way to test or fix things after merging. You mention it changes stuff to use switch statements. I had to remove those at the start of the project due to python environment differences of the users. I am all for yolo’ing stuff to prod, but I’d like to be there when stuff fails to quickly pick up the issues. Especially when the changes are this huge. |
85s2ksnw4v-commits
approved these changes
Nov 12, 2025
85s2ksnw4v-commits
approved these changes
Nov 12, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hey I added tox to the project and added isort and flake8/pep8.
Isort will sort all import in a standard format.
flake8/pep8 add a python code style guide.
To run the text you need to install tox
pip install toxThen run it with
toxwill run isort and pep8Examples: